[REFACTOR] /noti 워크플로우 수정 #13
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
leegwichan
approved these changes
Dec 10, 2024
Comment on lines
77
to
81
| run: | | ||
| echo "AVATAR_URL=${{ secrets.DISCORD_AVATAR_URL }}" >> $GITHUB_ENV | ||
| echo "COMMENT_BODY= 🤚백엔드 친구들 모여라~ 🔊" >> $GITHUB_ENV | ||
| echo "USERNAME=망나뇽" >> $GITHUB_ENV | ||
| echo "WEB_HOOK=${{ secrets.DISCORD_WEB_HOOK }}" >> $GITHUB_ENV |
There was a problem hiding this comment.
[제안]
- "망나뇽" 딴 걸로 바꾸실?
- COMMENT_BODY 안 넣는게 오히려 좋을 수도?
- 지금 알림을 보면 '어 이거(🤚백엔드 친구들 모여라~ 🔊) 왜있지?' 라는 생각이 들어서
There was a problem hiding this comment.
[커찬의 제안에 대한 의견]
- 망나뇽 바꾸는 것 찬성합니다 ㅋㅋㅋ
디코봇정도 네이밍은 어떠신가요? - 저도 유의한 comment body는 넣어야 한다는 것에 찬성해요. 지금은 어떤 PR에 코멘트를 남겼는지 구분이 되지 않는다는 단점이 있었던 것 같아서, PR 타이틀을 넣어보는 건 어떨까요?
Suggested change
| run: | | |
| echo "AVATAR_URL=${{ secrets.DISCORD_AVATAR_URL }}" >> $GITHUB_ENV | |
| echo "COMMENT_BODY= 🤚백엔드 친구들 모여라~ 🔊" >> $GITHUB_ENV | |
| echo "USERNAME=망나뇽" >> $GITHUB_ENV | |
| echo "WEB_HOOK=${{ secrets.DISCORD_WEB_HOOK }}" >> $GITHUB_ENV | |
| run: | | |
| echo "AVATAR_URL=${{ secrets.DISCORD_AVATAR_URL }}" >> $GITHUB_ENV | |
| echo "COMMENT_BODY= github.event.issue.pull_request.title 리뷰 완료!" >> $GITHUB_ENV | |
| echo "USERNAME=디코봇(더 좋은 이름이 있다면 그걸로!)" >> $GITHUB_ENV | |
| echo "WEB_HOOK=${{ secrets.DISCORD_WEB_HOOK }}" >> $GITHUB_ENV |
There was a problem hiding this comment.
디코봇, 해당 PR or Issue의 타이틀로 수정했습니다!
|
|
||
| jobs: | ||
| notify-discord: | ||
| notify-discord-comment: |
There was a problem hiding this comment.
오... 자동화 actions를 자세히 뜯어보는 건 오늘이 처음인데, 그럼 꼭 상단에 /noti 를 적지 않아도 트리거가 발동되겠군요! 저는 지금까지 상단에 적어야 하는 건 줄 알았어요 😅
There was a problem hiding this comment.
그냥 들어있기만 하면 상관없습니다!
Comment on lines
77
to
81
| run: | | ||
| echo "AVATAR_URL=${{ secrets.DISCORD_AVATAR_URL }}" >> $GITHUB_ENV | ||
| echo "COMMENT_BODY= 🤚백엔드 친구들 모여라~ 🔊" >> $GITHUB_ENV | ||
| echo "USERNAME=망나뇽" >> $GITHUB_ENV | ||
| echo "WEB_HOOK=${{ secrets.DISCORD_WEB_HOOK }}" >> $GITHUB_ENV |
There was a problem hiding this comment.
[커찬의 제안에 대한 의견]
- 망나뇽 바꾸는 것 찬성합니다 ㅋㅋㅋ
디코봇정도 네이밍은 어떠신가요? - 저도 유의한 comment body는 넣어야 한다는 것에 찬성해요. 지금은 어떤 PR에 코멘트를 남겼는지 구분이 되지 않는다는 단점이 있었던 것 같아서, PR 타이틀을 넣어보는 건 어떨까요?
Suggested change
| run: | | |
| echo "AVATAR_URL=${{ secrets.DISCORD_AVATAR_URL }}" >> $GITHUB_ENV | |
| echo "COMMENT_BODY= 🤚백엔드 친구들 모여라~ 🔊" >> $GITHUB_ENV | |
| echo "USERNAME=망나뇽" >> $GITHUB_ENV | |
| echo "WEB_HOOK=${{ secrets.DISCORD_WEB_HOOK }}" >> $GITHUB_ENV | |
| run: | | |
| echo "AVATAR_URL=${{ secrets.DISCORD_AVATAR_URL }}" >> $GITHUB_ENV | |
| echo "COMMENT_BODY= github.event.issue.pull_request.title 리뷰 완료!" >> $GITHUB_ENV | |
| echo "USERNAME=디코봇(더 좋은 이름이 있다면 그걸로!)" >> $GITHUB_ENV | |
| echo "WEB_HOOK=${{ secrets.DISCORD_WEB_HOOK }}" >> $GITHUB_ENV |
|
/noti 비토, review용 comment와 일반 코멘트의 워크플로를 모두 트리거로 지정해주어 이젠 두 상황 모두에서 코멘트 웹 훅이 트리거되겠네요! 사소한 부분이었는데 신경써주어 감사합니다 🙇 커찬 의견과 관련하여 사소한 의견 남겼으니 반영은 비토가 자유롭게 해주세요! |
|
/noti |
|
/noti |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🚩 연관 이슈
closed #12
🗣️ 리뷰 요구사항 (선택)
job을 리뷰코멘트용이랑 코멘트용으로 분리했습니다